Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for wound overlays. #5787

Merged
merged 1 commit into from
Feb 23, 2024
Merged

Fix for wound overlays. #5787

merged 1 commit into from
Feb 23, 2024

Conversation

Segrain
Copy link
Contributor

@Segrain Segrain commented Feb 23, 2024

About the pull request

Yet another disastrous consequence of that "research warcrimes" PR. For whatever reason, it removed a line that actually was doing something important. If we have been processing the limb (otherwise need_process() should not be called at all) and after going through all the checks decided to cease processing it (which only happens when all damage is gone), we do need to update wounds one final time first to remove all the overlays from wounds/bandages that have healed but have not been removed by limb's not-every-tick update in regular process().

Explain why it's good for the game

Fixes #4764.

Changelog

🆑
fix: Fully healed limbs should now properly remove the overlays for their wounds and bandages.
/:cl:

@github-actions github-actions bot added the Fix Fix one bug, make ten more label Feb 23, 2024
@harryob harryob added this pull request to the merge queue Feb 23, 2024
Merged via the queue into cmss13-devs:master with commit b9ae5b4 Feb 23, 2024
28 checks passed
cm13-github added a commit that referenced this pull request Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sprite damage doesn't resolve after healing
2 participants